home *** CD-ROM | disk | FTP | other *** search
/ Amiga Tools 3 / Amiga Tools 3.iso / grafik / raytracing / rayshade-4.0.6.3 / fixes / fix026.shadowbug < prev    next >
Encoding:
Text File  |  1994-08-09  |  4.0 KB  |  166 lines
  1. Intersect bug:
  2. AddToHitList adds the object to the hitlist without checking
  3. if there is room in the hitlist->data array.
  4.  
  5. Shadowcache bug:
  6. There is a problem if a CSG object or an animated object is
  7. put in the shadowcache. If the cached object is used in the
  8. next Shadowed() call, TRUE is returned if the object is hit.
  9. This can be wrong, because parts of the object may be transparent.
  10. The following scene will show the bug:
  11.  
  12. surface glass
  13. ambient .02 .02 .02
  14. diffuse 0.1 0.1 0.1
  15. specular 0.8 0.8 0.8 specpow 200
  16. transp 1. index 1.2 extinct 0.98
  17.  
  18. screen 200 150
  19. eyep -20 30 10
  20. lookp 0 0 0
  21. fov 50
  22. light 1 directional -30 0 50
  23. background .3 0 .4
  24. shutter 1
  25.  
  26. list
  27. sphere glass 3  3 0 -3
  28. sphere  3  3 0  6
  29. end
  30. translate (time) 0 0
  31. plane 0 0 -12 0 0 1
  32.  
  33.  
  34. ======================= cut here ========================
  35.  
  36. *** intersect.c.orig    Wed Jun 17 17:36:47 1992
  37. --- intersect.c    Wed Jun 17 17:38:51 1992
  38. ***************
  39. *** 139,146 ****
  40.   {
  41.       HitNode *np;
  42.       Trans *list;
  43.   
  44. !     np = &hitlist->data[hitlist->nodes++];
  45.   
  46.       np->ray = *ray;
  47.       np->obj = obj;
  48. --- 139,150 ----
  49.   {
  50.       HitNode *np;
  51.       Trans *list;
  52. +     int i;
  53.   
  54. !     np = &hitlist->data[i = hitlist->nodes++];
  55. ! 
  56. !     if (i >= MAXMODELDEPTH)
  57. !         RLerror(RL_ABORT, "HitList full. Increase libray/libobj/geom.h:MAXMODELDEPTH.\n");
  58.   
  59.       np->ray = *ray;
  60.       np->obj = obj;
  61. *** light.h.orig    Thu Jun 18 15:15:04 1992
  62. --- light.h    Thu Jun 18 15:18:16 1992
  63. ***************
  64. *** 43,50 ****
  65.   
  66.   typedef struct {
  67.       struct Geom *obj;    /* Pointer to cached object */
  68. !     RSMatrix trans;    /* World-to-object transformation */
  69.       char dotrans;        /* TRUE if above trans is non-identity */
  70.   } ShadowCache;
  71.   
  72.   typedef struct {
  73. --- 43,52 ----
  74.   
  75.   typedef struct {
  76.       struct Geom *obj;    /* Pointer to cached object */
  77. !     RSMatrix trans;        /* World-to-object transformation */
  78.       char dotrans;        /* TRUE if above trans is non-identity */
  79. +     char transp;        /* TRUE if parts of the object that */
  80. +                 /* have no surface will be transparent */
  81.   } ShadowCache;
  82.   
  83.   typedef struct {
  84. *** shadow.c.orig    Mon Jun 15 18:37:18 1992
  85. --- shadow.c    Thu Jun 18 17:25:51 1992
  86. ***************
  87. *** 94,107 ****
  88.                */
  89.               if (intersect(cp->obj, &tmpray, &hitlist,
  90.                   SHADOW_EPSILON, &s)) {
  91. !                 CacheHits++;
  92. !                 return TRUE;
  93.               }
  94.           } else if (IsAggregate(cp->obj)) {
  95.               if ((*cp->obj->methods->intersect)(cp->obj->obj,
  96.                   &tmpray, &hitlist, SHADOW_EPSILON, &s)) {
  97. !                 CacheHits++;
  98. !                 return TRUE;
  99.               }
  100.           } else if ((*cp->obj->methods->intersect)(cp->obj->obj,
  101.                       &tmpray, SHADOW_EPSILON, &s)) {
  102. --- 94,118 ----
  103.                */
  104.               if (intersect(cp->obj, &tmpray, &hitlist,
  105.                   SHADOW_EPSILON, &s)) {
  106. !                 /* Cannot use GetShadingSurf() */
  107. !                 for (sptr = 0, i = 0; i < hitlist.nodes; i++)
  108. !                     if (sptr = hitlist.data[i].obj->surf)
  109. !                         break;
  110. !                 if (sptr && sptr->transp < EPSILON || sptr == 0 && !cp->transp) {
  111. !                         CacheHits++;
  112. !                         return TRUE;
  113. !                 }
  114.               }
  115.           } else if (IsAggregate(cp->obj)) {
  116.               if ((*cp->obj->methods->intersect)(cp->obj->obj,
  117.                   &tmpray, &hitlist, SHADOW_EPSILON, &s)) {
  118. !                 for (sptr = 0, i = 0; i < hitlist.nodes; i++)
  119. !                     if (sptr = hitlist.data[i].obj->surf)
  120. !                         break;
  121. !                 if (sptr && sptr->transp < EPSILON || sptr == 0 && !cp->transp) {
  122. !                         CacheHits++;
  123. !                         return TRUE;
  124. !                 }
  125.               }
  126.           } else if ((*cp->obj->methods->intersect)(cp->obj->obj,
  127.                       &tmpray, SHADOW_EPSILON, &s)) {
  128. ***************
  129. *** 252,257 ****
  130. --- 263,269 ----
  131.       HitNode *np;
  132.       int i, n;
  133.       extern long ShadowOptions;
  134. +     int gotsurf = FALSE;
  135.   
  136.       i = 0;
  137.   
  138. ***************
  139. *** 295,300 ****
  140. --- 307,316 ----
  141.           i++;
  142.           np++;
  143.       }
  144. +     /*
  145. +      * DefaultSurface is not transparent
  146. +      */
  147. +     cache->transp = FALSE;
  148.       cache->dotrans = FALSE;
  149.       while (i < hitlist->nodes -1) {
  150.           if (np->obj->trans) {
  151. ***************
  152. *** 309,314 ****
  153. --- 325,335 ----
  154.                       &cache->trans);
  155.                   cache->dotrans = TRUE;
  156.               }
  157. +         }
  158. +         if (!gotsurf && np->obj->surf) {
  159. +             gotsurf = TRUE;
  160. +             if (np->obj->surf->transp >= EPSILON)
  161. +                 cache->transp = TRUE;
  162.           }
  163.           i++;
  164.           np++;
  165.  
  166.